Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace styles on new-user form #1706

Merged
merged 2 commits into from
May 20, 2016
Merged

Conversation

jrjohnson
Copy link
Member

This is a new structured approach to styling components. For now I'm
placing the files in newcomponents/ directory but once we have replaced
styles on the rest of the site the components/ directors will be
removed.

Individual ember components get a class and then we use mixins as
functions to apply the actual styles to the page. Hopefully this gives
us a bunch of re-uability without needing to put things in a global
style space at all.

Refs #1705

This is a new structured approach to styling components.  For now I'm
placing the files in newcomponents/ directory but once we have replaced
styles on the rest of the site the components/ directors will be
removed.

Individual ember components get a class and then we use mixins as
functions to apply the actual styles to the page.  Hopefully this gives
us a bunch of re-uability without needing to put things in a global
style space at all.
new-user needed access to the cohort - no idea why it was running
before, probably an artifact of running after acceptance tests.

new-directory-user was actually testing new-user.
@stopfstedt stopfstedt merged commit c9cc475 into ilios:master May 20, 2016
@jrjohnson jrjohnson deleted the 1705-compactstyles branch May 20, 2016 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants